-
Notifications
You must be signed in to change notification settings - Fork 350
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: Revise release note for profile option and add example to the certificate usage section #1407
docs: Revise release note for profile option and add example to the certificate usage section #1407
Conversation
…ertificate usage section Signed-off-by: Peter Fiddes <peter.fiddes@gmail.com>
Signed-off-by: Peter Fiddes <peter.fiddes@gmail.com>
✅ Deploy Preview for cert-manager-website ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for cert-manager-website ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Peter.
I spotted a typo by otherwise lgtm.
content/docs/usage/certificate.md
Outdated
@@ -96,6 +96,15 @@ spec: | |||
# This is optional since cert-manager will default to this value however | |||
# if you are using an external issuer, change this to that issuer group. | |||
group: cert-manager.io | |||
|
|||
# keystores allows additing additional output formats. This is an example reference only. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# keystores allows additing additional output formats. This is an example reference only. | |
# keystores allows adding additional output formats. This is an example for reference only. |
@@ -57,7 +57,8 @@ so that clients can verify the identity of the metrics server. | |||
|
|||
The liveness probe of the cert-manager controller Pod is now enabled by default. | |||
|
|||
There is a new option `.spec.keystores.pkcs12.algorithms` to specify encryption and MAC algorithms for PKCS. | |||
There is a new option `.spec.keystores.pkcs12.profile` to specify encryption and HMAC algorithms for PKCS keystores. | |||
See the [API reference](../../../docs/reference/api-docs/#cert-manager.io/v1.PKCS12Profile) for configuration options. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you need to link to the markdown document. See:
website/content/docs/configuration/acme/http01/README.md
Lines 51 to 53 in f91577c
The HTTP01 Issuer supports a number of additional options. For full details on the range of options available, read the [reference documentation](../../../reference/api-docs.md#acme.cert-manager.io/v1.ACMEChallengeSolverHTTP01).
Signed-off-by: Peter Fiddes <peter.fiddes@gmail.com>
…the url Signed-off-by: Peter Fiddes <peter.fiddes@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: wallrj The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Fixes #1406.
/cc @wallrj